Skip to content

Add Nuget Version support in Univers - #60

Merged
TG1999 merged 2 commits into
aboutcode-org:mainfrom
TG1999:nuget_version_range
Apr 15, 2022
Merged

Add Nuget Version support in Univers#60
TG1999 merged 2 commits into
aboutcode-org:mainfrom
TG1999:nuget_version_range

Conversation

@TG1999

@TG1999 TG1999 commented Apr 12, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

@TG1999

TG1999 commented Apr 12, 2022

Copy link
Copy Markdown
Contributor Author

TODO:
Add ABOUT file for Nuget code
Add NugetVersion Range and add tests for it

@TG1999
TG1999 force-pushed the nuget_version_range branch 2 times, most recently from cf82f9a to 3e89f8c Compare April 13, 2022 12:16

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! ... here are some nits for your consideration!

Comment thread src/univers/nuget.py.ABOUT Outdated
Comment thread src/univers/nuget.py.ABOUT Outdated
Comment thread src/univers/nuget.py.NOTICE
Comment thread src/univers/nuget.py Outdated
Comment thread src/univers/nuget.py
@TG1999
TG1999 force-pushed the nuget_version_range branch from 3e89f8c to 6d9badc Compare April 13, 2022 18:40
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 force-pushed the nuget_version_range branch from 6d9badc to 725797c Compare April 13, 2022 18:47
Comment thread tests/test_nuget.py
@@ -0,0 +1,77 @@
# Copyright 2022 Google LLC
#

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding the missing copyright from upstream https://github.com/NuGet/NuGet.Client/blob/eea4636696cc7b6a3e682797c3dd717ad25fd7c9/test/NuGet.Core.Tests/NuGet.Versioning.Test/VersionComparerTests.cs

Suggested change
#
# Copyright (c) .NET Foundation. All rights reserved.

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 merged commit e16a494 into aboutcode-org:main Apr 15, 2022
TG1999 pushed a commit to TG1999/univers that referenced this pull request May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants